Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add chia services group #19044

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from
Draft

add chia services group #19044

wants to merge 8 commits into from

Conversation

altendky
Copy link
Contributor

@altendky altendky commented Dec 13, 2024

Purpose:

we all know that i don't like if __name__ == "__main__":, running python files directly, having runnable things not included in the command tree of chia, having to use the daemon to launch services, inconsistent usability of the cli to specify the root path, and... well, maybe that's enough.

this eliminates execution of services like python chia/server/start_full_node.py or python -m chia.server.start_full_node in favor of chia services full-node or python -m chia services full-node. Since the services are directly executed within the process, rather than launched via the daemon, they are still able to be launched by a debugger as needed or by a service manager like systemd. the entry point chia_full_node is retained (for now... :]).

Current Behavior:

New Behavior:

Testing Notes:

Draft For:

  • conversations
  • conversion of other services

@altendky altendky added Added Required label for PR that categorizes merge commit message as "Added" for changelog Changed Required label for PR that categorizes merge commit message as "Changed" for changelog and removed Added Required label for PR that categorizes merge commit message as "Added" for changelog labels Dec 13, 2024
Base automatically changed from chia_command_without_group to main December 13, 2024 18:49
@github-actions github-actions bot added the merge_conflict Branch has conflicts that prevent merge to main label Dec 13, 2024
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot removed the merge_conflict Branch has conflicts that prevent merge to main label Dec 13, 2024
Copy link
Contributor

Conflicts have been resolved. A maintainer will review the pull request shortly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changed Required label for PR that categorizes merge commit message as "Changed" for changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant